Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Turn on checksum validation for CRT S3 transfer manager #280

Merged
merged 1 commit into from
Nov 13, 2023

Conversation

kyleknap
Copy link
Contributor

For uploads, the CRT S3 client will add CRC32 trailing checksums. For downloads, the CRT S3 client will validate checksums associated to the object when possible.

This is a port from this CLI v2 PR: aws/aws-cli#8298

For uploads, the CRT S3 client will add CRC32 trailing checksums.
For downloads, the CRT S3 client will validate checksums
associated to the object when possible.
@kyleknap kyleknap requested a review from nateprewitt November 13, 2023 22:01
@codecov-commenter
Copy link

Codecov Report

Attention: 13 lines in your changes are missing coverage. Please review.

Comparison is base (b8906b3) 84.97% compared to head (d85a0fa) 84.57%.

Files Patch % Lines
s3transfer/crt.py 0.00% 13 Missing ⚠️

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@                 Coverage Diff                  @@
##           crt-enhancements     #280      +/-   ##
====================================================
- Coverage             84.97%   84.57%   -0.41%     
====================================================
  Files                    16       16              
  Lines                  2736     2749      +13     
====================================================
  Hits                   2325     2325              
- Misses                  411      424      +13     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Contributor

@nateprewitt nateprewitt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@kyleknap kyleknap merged commit db20da4 into boto:crt-enhancements Nov 13, 2023
13 checks passed
@kyleknap kyleknap deleted the crt-checksums branch November 13, 2023 22:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants